Skip to content

More various fixes (casting, missing checks etc...)#20

Merged
danielinux merged 10 commits into
wolfSSL:masterfrom
gasbytes:fixes-2
Feb 9, 2026
Merged

More various fixes (casting, missing checks etc...)#20
danielinux merged 10 commits into
wolfSSL:masterfrom
gasbytes:fixes-2

Conversation

@gasbytes

@gasbytes gasbytes commented Feb 6, 2026

Copy link
Copy Markdown
Contributor

Detected while fuzzing it through sequence (single packets, multiple, and more mutations).

- validate TCP header langth against IP payload and see if it actually
  fits or not, return on error
…mbers, preventing unsigned wraparound in bytes_in_flight
…cksum update

- add tcp_seq_inc() helper for wrap-safe sequence number arithmetic,
  replacing bare ee32(tcp->seq) + 1 in all five call sites
- fold carry in ICMP echo reply checksum update to avoid uint16_t truncation
@gasbytes gasbytes self-assigned this Feb 6, 2026
@gasbytes gasbytes assigned danielinux and unassigned gasbytes Feb 6, 2026
@gasbytes gasbytes requested a review from danielinux February 6, 2026 16:24
@danielinux danielinux merged commit 2fd11af into wolfSSL:master Feb 9, 2026
4 checks passed
@gasbytes gasbytes deleted the fixes-2 branch February 13, 2026 17:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants